Skip to content

Enable SwiftLint rule: toggle_bool#25504

Open
mokagio wants to merge 1 commit intotrunkfrom
mokagio/toggle_bool
Open

Enable SwiftLint rule: toggle_bool#25504
mokagio wants to merge 1 commit intotrunkfrom
mokagio/toggle_bool

Conversation

@mokagio
Copy link
Copy Markdown
Contributor

@mokagio mokagio commented Apr 21, 2026

Summary

Enables the toggle_bool SwiftLint rule, which prefers someBool.toggle() over someBool = !someBool.

  • Auto-fixed violations: 12 (across 7 files)
  • Manual fixes: 0
  • Violations left for human review: none

Part of the Orchard SwiftLint rollout campaign.


Posted by Claude (Opus 4.7) on behalf of @mokagio with approval.

Prefer `someBool.toggle()` over `someBool = !someBool`. 12 violations
auto-fixed; no manual changes required.

Part of the Orchard SwiftLint rollout campaign.
@mokagio mokagio self-assigned this Apr 21, 2026
@wpmobilebot
Copy link
Copy Markdown
Contributor

App Icon📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress
ConfigurationRelease-Alpha
Build Number32040
VersionPR #25504
Bundle IDorg.wordpress.alpha
Commit55eb8ce
Installation URL3pi80n73t80t0
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot
Copy link
Copy Markdown
Contributor

App Icon📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack
ConfigurationRelease-Alpha
Build Number32040
VersionPR #25504
Bundle IDcom.jetpack.alpha
Commit55eb8ce
Installation URL58qum6dind81o
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@mokagio mokagio added this to the 26.9 milestone Apr 22, 2026
@mokagio mokagio requested review from crazytonyli and jkmassel April 23, 2026 05:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants